-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add begining of docs for editor #144
base: master
Are you sure you want to change the base?
Conversation
The albinos configuration editor works in a complementary way with `albinos daemon service`. | ||
|
||
The albinos editor binary comes with a dynamic library containing albinos API functionality, | ||
This way when a client enters a command the function is directly called from the `DLL`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I4d object to the use of DLL since that may be confusing as it is primarly used for windows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is there a .idea
directory?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mergeable in the current state, nevertheless here are a few comments
Below is the list of prerequisites to compile albinos-editor on your machine: | ||
|
||
* Nim version 0.18 minimum | ||
* Nimble |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add instructions on how to get this. Isn't as straightforward as you make it out to be.
This is mostly because as far I know most package managers don't provide nimble.
A simple link to the official download or something like that would suffice.
|
||
## Getting Started | ||
|
||
These instructions will get you a copy of the project up and running on your local machine for development and testing purposes. See deployment for notes on how to deploy the project on a live system. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is "deployment"?
I Will still update this pull request, please dont merge |
No description provided.